home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 536 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  1.0 KB
  1. Path: news.clark.net!not-for-mail
  2. From: eamick@clark.net (Eric Amick)
  3. Newsgroups: comp.lang.c
  4. Subject: Re: Problem with stringcopy
  5. Date: 6 Jan 1996 17:44:30 GMT
  6. Organization: the end of my rope
  7. Message-ID: <4cmcdu$bm5@clarknet.clark.net>
  8. References: <4clguu$9fs@eagle.novo.dk> <yewvimppjz5.fsf@hyll.idt.unit.no> <4cmafq$bm5@clarknet.clark.net>
  9. NNTP-Posting-Host: explorer.clark.net
  10. Mime-Version: 1.0
  11. Content-Type: TEXT/PLAIN; charset=ISO-8859-1
  12. Content-Transfer-Encoding: 8bit
  13. X-Newsreader: TIN [UNIX 1.3 950726BETA PL0]
  14.  
  15. Eric Amick (eamick@clark.net) wrote:
  16. >>char *Stringcopy(char *source, int startpos, size_t length)
  17. >>{
  18. >>    char tmpstring[SIZEBIGENOUGH];
  19. >>    strncpy((char *)(source+startpos),tmpstring,length);
  20. >>    return tmpstring;
  21. >>}
  22. >
  23. >This won't work as written; the first two arguments to strncpy() are 
  24. >reversed.  The cast is also unnecessary.
  25.  
  26. Not to mention that returning a pointer to a local array is a bad idea.  
  27. I *KNEW* there was something I missed... ;-)
  28.  
  29. -- 
  30. Eric Amick            eamick@clark.net
  31. Columbia, MD            Public key available via finger
  32.